Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[2.0.3] 피드백 최소글자 수정 및 토큰 키 값 수정 #209

Merged
merged 4 commits into from
May 28, 2024
Merged

[2.0.3] 피드백 최소글자 수정 및 토큰 키 값 수정 #209

merged 4 commits into from
May 28, 2024

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented May 20, 2024 •
edited
Loading

내용

  • 서버에서 피드백 최소 글자가 5글자 이상으로 수정

    • AS-IS: 4글자
    • TO-BE: 5글자

  • 키 값 관리에 대한 이슈

    • 기존 키 값 관리를 각 모듈에서 스트링 값으로 독립적으로 관리하고 있습니다.
    • 문제점: fcmToken 값을 각각 관리해서 키 값이 변동된 다른 쪽에 싱크가 맞지 않아서 문제 발생
    • 근본적인 해결: 모든 장소에서 하나의 값을 바라보도록 구조 수정
    • 핫픽스 버전에서의 수정: 우선 키 값만 맞춰줌

@lgvv lgvv self-assigned this May 20, 2024
@lgvv lgvv added 버그 쿠링 쪽 버그 더보기 더보기 기능 (피드백 포함) labels May 20, 2024
@lgvv
Copy link
Member Author

lgvv commented May 20, 2024

/쿠링 패키지 빌드

@github-actions GitHub Actions
Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

@github-actions GitHub Actions
Copy link

✅ 패키지를 성공적으로 빌드했습니다.

sunshiningsoo
sunshiningsoo approved these changes May 20, 2024
View reviewed changes
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫡

@lgvv
Copy link
Member Author

lgvv commented May 28, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label May 28, 2024
@github-actions github-actions bot merged commit 17a4449 into develop/2.0.0 May 28, 2024
@github-actions github-actions bot deleted the feature/lgvv/fcm-key-sync branch May 28, 2024 12:32
@github-actions GitHub Actions
Copy link

🕊️ PR #209이 머지되었습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunshiningsoo sunshiningsoo sunshiningsoo approved these changes

@wonniiii wonniiii Awaiting requested review from wonniiii

Assignees

@lgvv lgvv

Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 더보기 더보기 기능 (피드백 포함) 버그 쿠링 쪽 버그
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lgvv @sunshiningsoo